gdk: Depend less on gdk_event_get_device()
authorCarlos Garnacho <carlosg@gnome.org>
Tue, 23 Jun 2020 09:00:46 +0000 (11:00 +0200)
committerCarlos Garnacho <carlosg@gnome.org>
Tue, 23 Jun 2020 21:42:53 +0000 (23:42 +0200)
For the most part, we are interested in seats here.

gdk/gdksurface.c
gdk/wayland/gdkdevice-wayland.c
gdk/wayland/gdkprivate-wayland.h
gdk/wayland/gdksurface-wayland.c
gdk/x11/gdkeventsource.c

index 6eac19ac37feab3344030102e62f8b52ea44af25..96302ea63032080ccea8f2a6187e369d9fed2e5f 100644 (file)
@@ -2947,11 +2947,9 @@ gdk_surface_get_seat_from_event (GdkSurface *surface,
 {
   if (event)
     {
-      GdkDevice *device = gdk_event_get_device (event);
       GdkSeat *seat = NULL;
 
-      if (device)
-        seat = gdk_device_get_seat (device);
+      seat = gdk_event_get_seat (event);
 
       if (seat)
         return seat;
index fe4d767e4884b663da6c582d8d0ca30c69087bf8..524f17e277a74a307849800fe1f944c6badcdb28 100644 (file)
@@ -4886,10 +4886,9 @@ _gdk_wayland_display_remove_seat (GdkWaylandDisplay *display_wayland,
 }
 
 uint32_t
-_gdk_wayland_device_get_implicit_grab_serial (GdkWaylandDevice *device,
-                                              GdkEvent         *event)
+_gdk_wayland_seat_get_implicit_grab_serial (GdkSeat  *seat,
+                                            GdkEvent *event)
 {
-  GdkSeat *seat = gdk_device_get_seat (GDK_DEVICE (device));
   GdkEventSequence *sequence = NULL;
   GdkWaylandTouchData *touch = NULL;
 
index a824f7bb7df53854c6be8b3e5ea705ad9e9e7958..c38f9b7f6838da168e56337a255847e58f08524d 100644 (file)
@@ -134,8 +134,8 @@ void        _gdk_wayland_display_remove_seat    (GdkWaylandDisplay       *displa
                                                  guint32                  id);
 
 GdkKeymap *_gdk_wayland_device_get_keymap (GdkDevice *device);
-uint32_t _gdk_wayland_device_get_implicit_grab_serial(GdkWaylandDevice *device,
-                                                      GdkEvent   *event);
+uint32_t _gdk_wayland_seat_get_implicit_grab_serial(GdkSeat  *seat,
+                                                    GdkEvent *event);
 uint32_t _gdk_wayland_seat_get_last_implicit_grab_serial (GdkWaylandSeat     *seat,
                                                           GdkEventSequence **seqence);
 struct wl_data_device * gdk_wayland_device_get_data_device (GdkDevice *gdk_device);
index 60bfa9147d99c224a8ce3403758f0350f28c2e5c..8f8b791f88dcf1e94e4de9b594fd30f0e5d5c073 100644 (file)
@@ -3830,8 +3830,8 @@ gdk_wayland_surface_show_window_menu (GdkSurface *surface,
   GdkWaylandSurface *impl = GDK_WAYLAND_SURFACE (surface);
   GdkWaylandDisplay *display_wayland =
     GDK_WAYLAND_DISPLAY (gdk_surface_get_display (surface));
-  struct wl_seat *seat;
-  GdkWaylandDevice *device;
+  GdkSeat *seat;
+  struct wl_seat *wl_seat;
   double x, y;
   uint32_t serial;
 
@@ -3850,21 +3850,21 @@ gdk_wayland_surface_show_window_menu (GdkSurface *surface,
   if (!is_realized_toplevel (surface))
     return FALSE;
 
-  device = GDK_WAYLAND_DEVICE (gdk_event_get_device (event));
-  seat = gdk_wayland_device_get_wl_seat (GDK_DEVICE (device));
+  seat = gdk_event_get_seat (event);
+  wl_seat = gdk_wayland_seat_get_wl_seat (seat);
   gdk_event_get_position (event, &x, &y);
 
-  serial = _gdk_wayland_device_get_implicit_grab_serial (device, event);
+  serial = _gdk_wayland_seat_get_implicit_grab_serial (seat, event);
 
   switch (display_wayland->shell_variant)
     {
     case GDK_WAYLAND_SHELL_VARIANT_XDG_SHELL:
       xdg_toplevel_show_window_menu (impl->display_server.xdg_toplevel,
-                                     seat, serial, x, y);
+                                     wl_seat, serial, x, y);
       break;
     case GDK_WAYLAND_SHELL_VARIANT_ZXDG_SHELL_V6:
       zxdg_toplevel_v6_show_window_menu (impl->display_server.zxdg_toplevel_v6,
-                                         seat, serial, x, y);
+                                         wl_seat, serial, x, y);
       break;
     default:
       g_assert_not_reached ();
index 25338b4581b190376d4021ed5bb822113cb99413..ad0d6e05eeda8fd1c5264c0ca8953d33c176e152 100644 (file)
@@ -152,9 +152,8 @@ static void
 handle_touch_synthetic_crossing (GdkEvent *event)
 {
   GdkEventType evtype = gdk_event_get_event_type (event);
-  GdkDevice *device = gdk_event_get_device (event);
   GdkEvent *crossing = NULL;
-  GdkSeat *seat = gdk_device_get_seat (device);
+  GdkSeat *seat = gdk_event_get_seat (event);
   gboolean needs_enter, set_needs_enter = FALSE;
 
   if (quark_needs_enter == 0)
@@ -208,7 +207,7 @@ handle_touch_synthetic_crossing (GdkEvent *event)
 
   if (crossing)
     {
-      gdk_display_put_event (gdk_device_get_display (device), crossing);
+      gdk_display_put_event (gdk_seat_get_display (seat), crossing);
       gdk_event_unref (crossing);
     }
 }